docs: read the reviewer's status notice when a review loop is stuck#88
Merged
Conversation
…s command (resume vs review)
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThis PR synchronizes documentation across skill guides and build specifications to clarify handling when bot review becomes stalled. The workflow now directs users to check the reviewer's latest status notice and execute the appropriate CodeRabbit command ( ChangesReview Loop Bot-State Handling
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related issues
Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
thejustinwalsh
added a commit
that referenced
this pull request
May 24, 2026
…hase 4 gates + #80 Phase 5 (#32) was built against a pre-Phase-4 base; main since landed the skill-enforcement gates (#86), the positive-done-signal nudge loop (#80), the plan-comment guard, and the review-handling skill codification (#87/#88/#89). Integration (Phase 5 as the base, main's additions layered on): - driveOnce resolves a bare-stop through the #80 nudge loop (session stays alive during nudges); a ready, non-draft Epic PR resolves to `done`, which Phase 5 then parks on review-resolved. nudge-exhausted parks in waiting-human. - The plan-comment guard runs in driveOnce: an unposted `done` is demoted to `failed` so it never enters the review park. - DriveResult carries a DriveOutcome (StopClassification | nudge-exhausted); finalStateFor handles nudge-exhausted; the drive-step timeout widened for nudges. - Skill (4 copies) resolved to main's codified version; the #80 nudge + plan-comment tests rewritten park-aware. Verified: typecheck clean, full suite 289 pass / 0 fail, skills mirror in sync.
9 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Refines the "bump when stuck" guidance (skill review-loop step 4 + Phase 5 spec resume step). The naive rule — "comment
@coderabbitai review" — is wrong for the most common stall: CodeRabbit auto-pauses after an influx of commits (which the batch-and-push review loop itself produces), and a one-shot@coderabbitai reviewdoes not un-pause it. The bot posts a status notice naming the exact command.New rule: when a review is stuck, read the reviewer's status notice and run the command it names —
@coderabbitai resume(re-enable auto-review),@coderabbitai review(single pass),@coderabbitai resolve(close addressed threads). Don't guess.Caught live: #86's review was paused by our own loop's commit churn;
@coderabbitai resume(notreview) is what brought it back green.Skill change propagated across all four copies (canonical / mirror /
.claude/.codex); drift gate passes.Summary by CodeRabbit
@coderabbitai resume,@coderabbitai review,@coderabbitai resolve) to unblock workflows. Added clarity on distinguishing between paused and active review states.